gtk: Bubble drag events like motion events

Emit crossing events - with a new GTK_CROSSING_DROP type - like we do
for motion events. There is no more special casing for them.

Note that the gesture has not been updated yet, so some obscure behavior
may occur.

events: Use GdkDrop as event sequence

This allows treating drop events like touch events, which GTK groups by
event sequence.

It's a bit ugly that we just case the GdkDrop pointer, but event
sequences are only meant to be unique pointer ids, so it's fine.

widget: Simplify adjust_allocation()

And in particular, only do it if the widget doesn't use ALIGN_FILL.

This avoids lots of measuring in the common case and speeds up
size_allocate() by about 25%.

And because size_allocate() is the bottleneck in the fishbowl, this also
gets ~25% more fishies.

widget: Pull margin computation out of adjust_allocation

It's way cheaper to just do it.

Also simplifies adjust_size_allocation a lot.

sizerequest: Stop clamping for_size to natural size

Widgets should be given the actual size they will be allocated, so they
can do something with it.

If they want to clamp themselves to their natural size, nothing's
stopping them, they know their natural size after all.

widget: Don't check for natives

It's the native's job to request a 1px x 1px size, not the job of
gtk_widget_size_allocate()

Also saves 10% of size_allocate() time because checking for an interface
is really expensive.

Ensure all natives request at least a 1px wide surface

FIXME: Is this necessary?

Could the surfaces just clamp to 1x1 themselves?
We recently declared that surfaces can decide on whatever size they want
so natives need to inspect the size they requested anyway.

Merge branch 'color-float-fix' into 'master'

color chooser: Fix fallout from floatification

See merge request GNOME/gtk!1459

color chooser: Fix fallout from floatification

We are using (dddd) variants to store colors in variants,
which is dangerous now that GdkRGBA members are just floats.

Avoid passsing the GdkRGBA members directly to any varargs
functions.

Merge branch 'wip/xdg-popup-layout-no-op' into 'master'

gdk/wayland: Avoid relayout with the same properties

See merge request GNOME/gtk!1457

Merge branch 'wip/gdkpopuplayout-section' into 'master'

docs: Add GdkPopupLayout to gdk4-sections.txt

See merge request GNOME/gtk!1458

docs: Add GdkPopupLayout to gdk4-sections.txt

gdk/wayland: Avoid relayout with the same properties

When a popup is already showing, and gdk_surface_present_popup() is
called, if the layout didn't change, we're not really interested in
relayouting.

In the future, we'll be able to get notified if position of the popup
would change by some environmental changes, but until then, just don't
support it.

gdk/popup-layout: Remove leftover struct field

The layouts never "seal", as they did in an earlier revision, so remove
the seal field.

Merge branch 'wip/otte/for-master' into 'master'

surface: Don't take a display argument in gdk_surface_new_popup()

See merge request GNOME/gtk!1456

surface: Don't take a display argument in gdk_surface_new_popup()

The display can be taken from the surface, it must not be different.

Merge branch 'wip/otte/for-master' into 'master'

Wip/otte/for master

See merge request GNOME/gtk!1455

transform: Don't crash for gsk_transform_transform (id, id)

See attached tests

transform: Make sure the identity transform is equal to NULL

x11: When clearing old Drop, emit LEAVE event

This can happen when the old DND operation died (like due to a crash or
a broken XWayland compositor.

gdk: Make DRAG_ENTER event take x/y coordinates

Make it mirror the behavior of ENTER/LEAVE events.

transform: Add optimization for common case

Transforming identity by an other transform does not mean we need to
painstakingly apply the individual steps of other to construct a new
transform, it means we can just return other.

Or in math terms:
  I * B = B
so just return B.

Update POTFILES.in

profiling: Avoid one extra printf

We already format the message, no need to use
printf again to combine that with the kind string.

profiling: Avoid criticals

With events no longer GObjects, the type class is
longer around for peeking.

Merge branch 'readonly-events-1' into 'master'

Redo events

See merge request GNOME/gtk!1443

docs: Remove some no-longer existing api

Stop exporting gtk_get_event_widget

This is non-essential convenience API, and we
don't really expose events to applications anymore.

gtk-demo: Stop using gtk_get_event_widget

This is convenience api that will be going away.

gdk: Drop event structs from the headers

All events are GdkEvents now.

gtk: Stop using GtkEventButton

Same as the previous commit. All events are
just GdkEvents now.

gtk: Stop using GdkEventKey

We should just use GdkEvent, instead of taking
a GdkEventKey, and then cast it everywhere.

a11y: drop unused code

We are no longer using key snoopers.

motion controller: Match focus event propagation

Make the crossing event generation for pointer events
match what we do for focus now.

Keep more event controller api private

We don't want to expose the GtkCrossingData struct, and manually
feeding events to event controllers is not something we want to
encourage, going forward.

Stop exporting check_event_sanity

Bring back im context focus-in/out

The key controller still needs to track focus,
in order to emit these signals when required.

Go back to ::enter/::leave for pointer changes

These signals are behaving a little differently from
what ::focus-in/::focus-out used to do.

Split off GtkEventControllerFocus

Split the focus tracking into a separate
GtkEventControllerFocus, and change the API one more time.
We are back to having ::focus-in and ::focus-out signals.

Update all users.

wip: Add more information to crossing events

Add fields for direct descendents to GtkCrossingData,
and populate them when emitting focus change events.

Also add accessors for these fields to GtkEventControllerKey,
and verify that they are set properly in the focus test.

Not done yet: Do the same for pointer crossing events.

win32: Remove some leftover event struct access

And leftover old event api calls.

wayland: Fix a release build warning

Update the focus test

This needs an update to handle the new focus-change signal.

Update event docs section

Clean up GdkEventType docs

Some event struct packing improvements

Rearrange a few things, and move some booleans
into the Any struct, by using a bitfield there.

Some more cleanup could be done - the flags field
with its PENDING and FLUSHED members appears
entirely unused. Nobody is setting those flags.

Streamline event structs

Use proper types (GdkModifierType, double), and drop
some unused fields (send_event, display).

x11: Remove some leftover event struct access

x11: Stop using the send_event event field

The only use of this field is printing it out in a
debug message. We are going to drop it.

Always deliver focus events to toplevels

Its was GTK expects. This change gets rid of the "Ignoring an
unexpected focus event from GDK on a non-toplevel surface."
warning.

Drop gtk_widget_event from API

We don't want events to be injected randomly from
the outside. Plus, there's no way to create such
events from the outside now.

events: reorganize getters

Restructure the getters for event fields to
be more targeted at particular event types.

Update all callers, and replace all direct
event struct access with getters.

As a side-effect, this drops some unused getters.

Drop gdk_set/get_show_events

We don't need a separate api to turn on debugging
for events.

Strip const from GdkEvent

Events are refcounted structs, and we generally don't
pass these as const.

events: Drop all setters

GdkEvent is now truly readonly.

events: Drop target and related target

GTK no longer uses this, so we can drop it.

main: Stop calling gdk_event_set_target

GTK is no longer relying on this.

Stop using gdk_event_get_target

We can now get the target widget from the gesture
that we are using to find the event in the first
place.

gesture: Keep the target widget for events

We already store the events; keep the target widget
in addition. This is a step towards getting rid of
gdk_event_get_target.

eventcontroller: Make the target widget available

Make it possible for event controllers to obtain
the target widget during handle_event.

Explicitly pass the target to handle_event

Pass the event propagation target explicitly down to
the event controllers. This is a step towards getting
rid of gdk_event_set_target.

Stop looking at the related target for filtering

We are now sending crossing events (which are the only ones
where a related target makes sense) via handle_crossing.

Reinstate filtering for crossing events

The event propagation limit should apply to crossing events
as well.

Make crossing events handled the same way

New focus change handling

Instead of relying on gdk's antiquated crossing events,
create a new GtkCrossingData struct that contains the
actual widgets, and a new event controller vfunc that
expects this struct. This also saves us from making sense
of X's crossing modes and details, and makes for a
generally simpler api.

The ::focus-in and ::focus-out signals of GtkEventControllerKey
have been replaced by a single ::focus-change signal that
takes GtkCrossingData as an argument. All callers have
been updated.

wip: scrolledwindow stop using targets

main: Drop gtk_get_event_target

This is no longer used.

tooltip: stop using gtk_get_event_target

Stop using gtk_get_event_target

This is just a thin wrapper around gdk_event_get_target,
so use that directly.

Pass translated coordinates outside the event

We want to make events readonly, so stop translating
their coordinates and instead pass the translated
coordinates separately, when propagating events.

Drop GDK_NOTHING

Events of type GDK_NOTHING are good for nothing.

Make GdkEvent a boxed type

Stop using g_object_ref/unref on events

Use gdk_event_ref/unref instead of g_object_ref/unref.
Events will stop being object soon.

Drop gdk_event_new and gdk_event_copy

These functions are no longer used outside of gdkevents.c.

display: Stop using gdk_event_copy

Events are effectively readonly in GDK now, so we can just
take a reference, no need for a copy.

win32: Use event constructors

x11: Pass the right surface to the dnd filter

x11: Remove an unnecessary check

We are not creating GDK_NOTHING events anymore. Yay

x11: change event translator interface

Make the event translator return a new event, instead of
filling in a half-constructed one.

Update the two implementation in GdkX11Display and
GdkDeviceManagerXI2.

x11: Change the wm protocols filter api

Instead of passing a half-constructed event and expect
it to be filled in, pass the surface as in argument, and
add an out argument for a newly constructed GdkEvent.

x11: Don't pass a GdkEvent to shape cache filters

The filter functions never look at that event, and
we want to get out of the business of passing half-
constructed events around.

x11: Don't pass a GdkEvent to xsettings filters

The filter functions never look at that event, and
we want to get out of the business of passing half-
constructed events around.

events: Drop GDK_DESTROY

No backend is emitting GDK_DESTROY events anymore, so no
need to carry this around.

gtk: Stop handling GDK_DESTROY differently from GDK_DELETE

We don't have child windows anymore, so there is no difference.

x11: Don't generate GDK_DESTROY events

GTK does not differentiate between GDK_DESTROY and GDK_DELETE
anyway.

x11: use event constructors

This is not quite complete and only handles the
simple cases.

broadway: Use event constructors

Convert all of Broadways event handling to use the new
constructors.

wip: Use event constructors in the testsuite

This doesn't work because the constructors aren't exported,
so remove the gestures text temporarily, until we figure
out how to deal with it.

textview: Stop using gdk_event_copy

main: Use constructors instead of gdk_event_copy + rewriting

imcontextsimple: Use event constructors

widget: Port emulation code to constructors

Use the new event constructors when generating events
for emulating presses.

window: Use event constructors

main: Use event constructors

At the same time, stop setting child_surface in
crossing events. Nothing in GTK looks at it.

gdk: Use constructor for grab broken

gdk: Use event constructors for dnd

events: Drop x_root/y_root

We are not using these fields anymore.

broadway: Stop setting x_root/y_root in events

These fields are not used in GTK anymore, and are
going away soon.

x11: Stop setting x_root/y_root in events

These fields are not used by GTK anymore and
will be going away soon.

x11: Stop relying on root coordinates in events

The X11 dnd code was still using x_root/y_root in
GdkEventMotion in a couple of places. Stop doing so,
these field are going away soon.